Skip to content

ci: re-enable pr labeler #1296

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 10 commits into from
Closed

ci: re-enable pr labeler #1296

wants to merge 10 commits into from

Conversation

bowenli86
Copy link
Collaborator

Motivation:

Explain here the context, and why you're making that change. What is the problem you're trying to solve.

Modifications:

Describe the modifications you've done.

Result:

After your change, what will change.

bowenli86 added 10 commits April 1, 2025 11:33
**Motivation:**

chain_id is currently an github secrets, but not used anywhere

**Modifications:**

remove chain id from secrets and env vars 

**Result:**

one less useless secret, no impact on ci
**Motivation:**

per eigengit, main will be the new canonical branch

**Modifications:**

change push branch from dev to main

**Result:**

ci will be run on PRs against main
…Labs#1279)

**Motivation:**

fork-based pr cannot use secrets or github token

**Modifications:**

disable auto labeler to prepare for enabling fork-based pr

**Result:**

auto labeler is disabled, will be potentially removed in future soon
**Motivation:**

*Explain here the context, and why you're making that change. What is
the problem you're trying to solve.*

**Modifications:**

*Describe the modifications you've done.*

**Result:**

*After your change, what will change.*
**Motivation:**

remove holesky rpc since it's not used anywhere

**Modifications:**

remove holesky rpc

**Result:**

remove holesky rpc
**Motivation:**

make testing of fork based pr easier by splitting workflows require
secrets vs not

**Modifications:**

splitting workflows require secrets vs not into 2 gh workflow files

**Result:**

splitting workflows require secrets vs not into 2 gh workflow files
**Motivation:**

*Explain here the context, and why you're making that change. What is
the problem you're trying to solve.*

**Modifications:**

*Describe the modifications you've done.*

**Result:**

*After your change, what will change.*
@bowenli86 bowenli86 force-pushed the main branch 2 times, most recently from 9e7593b to 328a36d Compare April 15, 2025 21:34
@bowenli86 bowenli86 closed this Apr 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant